Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix the CI by adding a resolution for the broken release of enhanced resolve, and also overriding the dep version of types/node on sandboxes so that they may be linked #19936

Merged
merged 1 commit into from
Nov 23, 2022

Conversation

ndelangen
Copy link
Member

@ndelangen ndelangen commented Nov 23, 2022

Issue: the CI is currently failing, this should resolve that until webpack/enhanced-resolve#362 is fixed upstream

What I did

I added a yarn resolution to the script that generates the sandboxes.
This forces the use of a version of enhanced-resolve that we know works.

I added a install of @types/node version 16, that overrides the version of the generator, the ensure we can link the sanbox to storybook code via portals.

…resolve, and also overriding the dep version of types/node on sandboxes so that they may be linked
@ndelangen ndelangen self-assigned this Nov 23, 2022
@ndelangen ndelangen added the build Internal-facing build tooling & test updates label Nov 23, 2022
@storybookjs storybookjs deleted a comment from github-actions bot Nov 23, 2022
@ndelangen ndelangen merged commit 53af8af into next Nov 23, 2022
@ndelangen ndelangen deleted the fix/sandbox-script-bypass-ehanced-resolve-fire branch November 23, 2022 10:24
@tmeasday
Copy link
Member

@ndelangen can we revert this now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Internal-facing build tooling & test updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants